Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

Sign up for GitHub

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Jump to bottom

(maint) Fix ca-certificates update for el6; re-enable PDB #2589

Merged
merged 1 commit into from
Oct 6, 2021
Merged

(maint) Fix ca-certificates update for el6; re-enable PDB #2589

merged 1 commit into from
Oct 6, 2021

Conversation

Magisus
Copy link
Contributor

@Magisus Magisus commented Oct 6, 2021

This commit edits the yum repos on RHEL6 to allow us to install a newer
ca-certificates package that avoids the expired Let's Encrypt root
cert. This allows us to successfully install postgres, which in turn
allows this commit to also re-enable testing with PDB on EL6.

@Magisus
(maint) Fix ca-certificates update for el6; re-enable PDB
ef3c118 
This commit edits the yum repos on RHEL6 to allow us to install a newer
`ca-certificates` package that avoids the expired Let's Encrypt root
cert. This allows us to successfully install postgres, which in turn
allows this commit to also re-enable testing with PDB on EL6.
@Magisus Magisus requested a review from a team October 6, 2021 21:34
@justinstoller justinstoller merged commit 8c6d776 into puppetlabs:6.x Oct 6, 2021
@justinstoller
Copy link
Member

What happened with adding EL 8 again?

@Magisus
Copy link
Contributor Author

Magisus commented Oct 6, 2021

Still broken, at least on main. The DNF module support, which fixes it for 6.x, doesn't work on 7.x because they don't support Postgres 11 (just 10 and 12). Austin said he thought we were supposed to be doing something different: disabling postgres's repos in favor of RHEL's, which do have 11, but that doesn't appear to be working like he thought. See puppetlabs/puppetlabs-puppetdb#337 for details. We're not sure if this requires changes to the postgresql module, or just different configuration of it in the PDB module.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@mwaggett mwaggett mwaggett approved these changes

Assignees
No one assigned
Labels
None yet
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
3 participants
@Magisus @justinstoller @mwaggett